Skip to content

Conversation

@jr
Copy link
Collaborator

@jr jr commented Jun 23, 2025

Mostly to increase the odds we can debug settings fetch issues in Cursor.


Important

Enhance logging for settings fetch in CloudService and SettingsService to improve debugging capabilities.

  • Logging Enhancements:
    • Add log parameter to SettingsService constructor in SettingsService.ts for improved logging.
    • Update fetchSettings() in SettingsService.ts to use this.log for error and status messages.
  • CloudService Updates:
    • Pass this.log to SettingsService in CloudService.ts constructor.
  • Testing:
    • Update CloudService.test.ts to include log function in SettingsService constructor mock.

This description was created by Ellipsis for 708fb48. You can customize this summary. It will automatically update as commits are pushed.

@jr jr requested review from cte and mrubens as code owners June 23, 2025 20:21
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. enhancement New feature or request labels Jun 23, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jun 23, 2025
@mrubens mrubens merged commit 6670e1a into main Jun 23, 2025
21 checks passed
@mrubens mrubens deleted the jr/cloud-settings-logging-output branch June 23, 2025 20:26
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Jun 23, 2025
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Jun 23, 2025
cte pushed a commit that referenced this pull request Jun 24, 2025
Alorse pushed a commit to Alorse/Roo-Code that referenced this pull request Jun 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request lgtm This PR has been approved by a maintainer size:M This PR changes 30-99 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants